Repository navigation
Add restart IntelliSense command for active file - #4380
Closed
amritamishra01 wants to merge 2 commits into
Closed
amritamishra01 wants to merge 2 commits into
amritamishra01 wants to merge 2 commits into
Conversation
Contributor
|
Hey @amritamishra01 Thanks for the patch and the idea, but, I don't think we're going to go in this direction. Having the LSP glitch up does happen on occasion, but we usually present a popup to the user when it's needed (eg. a setting that won't take effect until the extension is restarted was changed). The "Restart and delete workspace" command, which is already available to end users, is more helpful for the cases where we don't explicitly prompt the user to restart (as an example of such a case: project was imported wrong and needs to be reimported). But thanks a lot for trying your hand at it! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #4293
Summary
This PR adds a new editor context menu action “Restart IntelliSense for Active File” for Java files.
The command appears when right-clicking inside a Java editor and reuses the existing Java language server restart flow to refresh IntelliSense.
Changes
java.runtime.restartIntellisensepackage.jsonsrc/extension.tsjava.server.restartcommand instead of duplicating restart logicTesting
Ran
npm run compilesuccessfullyVerified command registration in:
src/commands.tssrc/extension.tspackage.jsonLaunched Extension Development Host
Opened a
.javafileVerified the context menu shows Restart IntelliSense for Active File
Verified the command reuses the existing restart language server flow
Notes
Since the Java language server operates at the workspace level, this command is exposed only for the active Java editor while internally reusing the existing workspace-level restart behavior.